Skip to content

Fix portable binary paths, stale version pin, and missing SKILL.md in test suite - #2

Draft
ProjectAI00 with Copilot wants to merge 2 commits into
mainfrom
copilot/verify-loop-checks-cleanup
Draft

Fix portable binary paths, stale version pin, and missing SKILL.md in test suite#2
ProjectAI00 with Copilot wants to merge 2 commits into
mainfrom
copilot/verify-loop-checks-cleanup

Conversation

Copilot AI commented Mar 4, 2026 •
edited
Loading

Copy link
Copy Markdown

Test suite was broken in CI/non-author environments due to hardcoded absolute local paths, a stale version assertion, and a missing required asset.

Changes

  • tests/eval-loop.sh — Replace hardcoded /Users/aimar/... binary path with ${IMI_BIN:-<repo-relative>}, matching the existing pattern in integration.sh
  • tests/integration.sh — Same hardcoded fallback path fix; update version assertion from 0.3.4 → 0.3.19 to match Cargo.toml
  • skills/imi/SKILL.md — Create missing file (copied from npm/skills/imi/SKILL.md) required by section 23 of the integration suite; contains name, description, and allowed-tools frontmatter

Before/after binary resolution:

# Before (eval-loop.sh)
IMI_BIN="/Users/aimar/Documents/Kitchen/imi/ai-db-imi/target/release/imi"

# After
IMI_BIN="${IMI_BIN:-$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)/target/release/imi}"

Result: eval-loop.sh 5/5, integration.sh 79/79.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/ProjectAI00/ai-db-imi/releases/latest
    • Triggering command: /usr/bin/curl curl -s --max-time 5 -H Accept: application/vnd.github.v3&#43;json -H User-Agent: imi-cli REDACTED /home/REDACTED/work/ai-db-imi/ai-db-imi/target/release/deps/clap_derive-3a97accb5716ee�� /home/REDACTED/work/ai-db-imi/ai-db-imi/target/release/deps/clap_derive-3a97accb5716eef6.clap_derive.4aebc065d375203f-cgu.13.rcgu.o /home/REDACTED/work/ai-db-imi/ai-db-imi/target/release/deps/clap_derive-3a97accb5716eef6.clap_derive.4aebc065d375203f-cgu.14.rcgu.o (http block)
    • Triggering command: /usr/bin/curl curl -s --max-time 5 -H Accept: application/vnd.github.v3&#43;json -H User-Agent: imi-cli REDACTED 39025a3fc5b904.b-s id_mmbusjuz9lu8 ease/imi ent&#34;�� stable-x86_64-unPOST stable-x86_64-un-H rm&#34;:&#34;linux-x86_64&#34;,&#34;duration_ms&#34;:11,&#34;is_ci&#34;:true,&#34;interactive&#34;:false,&#34;install_id&#34;:&#34;iid_mmbusjuz stable-x86_64-uncurl stable-x86_64-un-s &#34;iid_mmbusjuz9l stable-x86_64-un/dev/null (http block)
    • Triggering command: /usr/bin/curl curl -s --max-time 5 -H Accept: application/vnd.github.v3&#43;json -H User-Agent: imi-cli REDACTED mi.c67e1696eb44c-s id_mmbusvpo9gl2 dirname t&#34;,&#34;distinct_id&#34;:&#34;uid_mmbusjv1qv79oz4a&#34;,&#34;properties&#34;:{&#34;version&#34;:&#34;0.3.19&#34;,&#34;platform&#34;:&#34;linux-x86_64&#34;,&#34;duration_ms&#34;:1,&#34;is_ci&#34;:true,&#34;interactive&#34;:false,&#34;install_id&#34;:&#34;iid_mmbusvpo9 mi.c67e1696eb44cPOST mi.c67e1696eb44c-H rm&#34;:&#34;linux-x86_64&#34;,&#34;duration_ms&#34;:20,&#34;is_ci&#34;:true,&#34;interactive&#34;:false,&#34;install_id&#34;:&#34;iid_mmbusvpo mi.c67e1696eb44ccurl hc9joo6s1yui9mjk-s &#34;iid_mmbusvpo9g k/ai-db-imi/ai-d/dev/null (http block)
  • us.i.posthog.com
    • Triggering command: /usr/bin/curl curl -s -o /dev/null -X POST -H Content-Type: application/json -d {&#34;api_key&#34;:&#34;phc_exyd1ppU0ZS7McQ1ay1gxvCaUI2QfYPuMCTk4kawVKF&#34;,&#34;event&#34;:&#34;imi_init&#34;,&#34;distinct_id&#34;:&#34;uid_mmbusjv1qv79oz4a&#34;,&#34;properties&#34;:{&#34;version&#34;:&#34;0.3.19&#34;,&#34;platform&#34;:&#34;linux-x86_64&#34;,&#34;duration_ms&#34;:31,&#34;is_ci&#34;:true,&#34;interactive&#34;:false,&#34;install_id&#34;:&#34;iid_mmbusjuz9lu --warn=unused_macro_rules --warn=unused_lifetimes 64-u�� 64-REDACTED-linux-gnu/lib/librustc_std_workspace_alloc-93ecc7719c6b3845.rlib 64-REDACTED-linux-gnu/lib/libminiz_oxide-ec844042f1e20091.rlib macro2-d920049c4/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/lib/rustlib/x86_gnu macro2-d920049c4/home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/rustc (dns block)
    • Triggering command: /usr/bin/curl curl -s -o /dev/null -X POST -H Content-Type: application/json -d {&#34;api_key&#34;:&#34;phc_exyd1ppU0ZS7McQ1ay1gxvCaUI2QfYPuMCTk4kawVKF&#34;,&#34;event&#34;:&#34;imi_init&#34;,&#34;distinct_id&#34;:&#34;uid_mmbusjv1qv79oz4a&#34;,&#34;properties&#34;:{&#34;version&#34;:&#34;0.3.19&#34;,&#34;platform&#34;:&#34;linux-x86_64&#34;,&#34;duration_ms&#34;:2,&#34;is_ci&#34;:true,&#34;interactive&#34;:false,&#34;install_id&#34;:&#34;iid_mmbusjuz9lu8 mi.c67e1696eb44c5ba-cgu.04.rcgu.o mi.c67e1696eb44c5ba-cgu.05.rcgu.o mi.c67e1696eb44c5ba-cgu.06.rcgu.o mi.c67e1696eb44c5ba-cgu.07.rcgu.o mi.c67e1696eb44c5ba-cgu.08.rcgu.o mi.c67e1696eb44c5ba-cgu.09.rcgu.o mi.c67e1696eb44c5ba-cgu.10.rcgu.o mi.c67e1696eb44c5ba-cgu.11.rcgu.o mi.c67e1696eb44c5ba-cgu.12.rcgu.o mi.c67e1696eb44c5ba-cgu.13.rcgu.o (dns block)
    • Triggering command: /usr/bin/curl curl -s -o /dev/null -X POST -H Content-Type: application/json -d {&#34;api_key&#34;:&#34;phc_exyd1ppU0ZS7McQ1ay1gxvCaUI2QfYPuMCTk4kawVKF&#34;,&#34;event&#34;:&#34;imi_goal&#34;,&#34;distinct_id&#34;:&#34;uid_mmbusjv1qv79oz4a&#34;,&#34;properties&#34;:{&#34;version&#34;:&#34;0.3.19&#34;,&#34;platform&#34;:&#34;linux-x86_64&#34;,&#34;duration_ms&#34;:2,&#34;is_ci&#34;:true,&#34;interactive&#34;:false,&#34;install_id&#34;:&#34;iid_mmbusjuz9lu8 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Task: Verify cleanup with full loop checks

Description

Run existing verification commands after cleanup and confirm no regressions in core loop behavior, writeback, and CLI flows. Record outcomes in completion summaries with exact command results.

Acceptance Criteria

Required tests pass and results are recorded in task completion summary.

Relevant Files

  • tests/eval-loop.sh
  • tests/integration.sh
  • src/main.rs

Tools

  • (none)

Goal description

Audit and remove dead code paths, unused modules, stale scripts, and unnecessary folders that are no longer part of the active IMI runtime. Prioritize safety by proving non-usage before deletion, preserving required skill/runtime assets, and keeping behavior unchanged for supported flows. Include a final verification pass to ensure CLI commands and test suites still pass after removals.

Goal why

Reduce maintenance burden, confusion, and risk from stale assets while keeping the core loop reliable.

Prior work on this goal

  • (none)

Decisions affecting this goal

  • (none)

Workspace Path

/Users/aimar/Documents/Kitchen/imi/ai-db-imi

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@ProjectAI00
Fix test infrastructure: portable binary paths, version check, missin…
d07456b 
…g SKILL.md

Co-authored-by: ProjectAI00 <174952601+ProjectAI00@users.noreply.github.com>
Copilot AI changed the title [WIP] Verify cleanup with full loop checks Fix portable binary paths, stale version pin, and missing SKILL.md in test suite Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@ProjectAI00 ProjectAI00

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ProjectAI00